Skip to content

270 enforce line endings#271

Merged
Enovotny merged 2 commits into
mainfrom
270-enforce-line-endings
Mar 31, 2026
Merged

270 enforce line endings#271
Enovotny merged 2 commits into
mainfrom
270-enforce-line-endings

Conversation

@krowvin

@krowvin krowvin commented Mar 9, 2026

Copy link
Copy Markdown
Collaborator

The .editorconfig will make it so the files save with specific EOL chars

the .gitattributes will make it so the files commit with specific EOL chars

This will happen for the entire repo.

I also ran git ls-files --eol to try and see if any files needed this applied and they all look to be in order!

@krowvin krowvin requested a review from Enovotny March 9, 2026 13:58
@krowvin krowvin linked an issue Mar 9, 2026 that may be closed by this pull request
@krowvin

krowvin commented Mar 9, 2026

Copy link
Copy Markdown
Collaborator Author

Same was done here
HydrologicEngineeringCenter/cwms-cli#144

Comment thread tests/cda/conftest.py Fixed
@krowvin krowvin force-pushed the 270-enforce-line-endings branch from e5eb908 to 010465d Compare March 9, 2026 14:11
@sonarqubecloud

sonarqubecloud Bot commented Mar 9, 2026

Copy link
Copy Markdown

@krowvin

krowvin commented Mar 12, 2026

Copy link
Copy Markdown
Collaborator Author

@Enovotny

The CI integration tests were cancelled, they're missing that "/" too

However, the PR'd changes are strictly for file EOL

@Enovotny Enovotny merged commit 6da9560 into main Mar 31, 2026
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enforce Line Endings

3 participants